-
-
Notifications
You must be signed in to change notification settings - Fork 398
chore: added json schema for configuration file #2068
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov ReportBase: 36.62% // Head: 36.56% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #2068 +/- ##
==========================================
- Coverage 36.62% 36.56% -0.07%
==========================================
Files 229 229
Lines 19539 19539
==========================================
- Hits 7156 7144 -12
- Misses 11547 11556 +9
- Partials 836 839 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
724b3e3
to
d012145
Compare
d012145
to
1f95eba
Compare
1f95eba
to
ed29087
Compare
ea69801
to
38231cd
Compare
c9eb6a4
to
8aa6453
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UPDATE: Resolved by 8ad388e
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UPDATE: superseded by 99eb639
7134c2a
to
34c68ce
Compare
34c68ce
to
99eb639
Compare
Co-authored-by: per1234 <[email protected]>
…to chore/json-schema
Co-authored-by: per1234 <[email protected]>
Co-authored-by: per1234 <[email protected]>
Co-authored-by: per1234 <[email protected]>
…to chore/json-schema
Co-authored-by: per1234 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Luca!
Please check if the PR fulfills these requirements
See how to contribute
before creating one)
our contributing guidelines
UPGRADING.md
has been updated with a migration guide (for breaking changes)What kind of change does this PR introduce?
Provides a json schema for the cli configuration file, for now the schema is not used by the runtime itself and it's only meant for users initiatives.
A possible followup of this pr is the autogeneration of the list of options in configuration.md
What is the current behavior?
What is the new behavior?
Does this PR introduce a breaking change, and is titled accordingly?
Other information
Fixes #587